Skip to content

Conversation

@jakkdl
Copy link
Member

@jakkdl jakkdl commented Dec 2, 2025

fixes #13734

yet another bandaid that would've been avoided if we had full support for exception groups in ExceptionInfo & friends 😶 see #9159

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided (automation) changelog entry is part of PR label Dec 2, 2025
Copy link
Member

@webknjaz webknjaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, short of one cosmetic improvement request.

@jakkdl
Copy link
Member Author

jakkdl commented Dec 8, 2025

I played around a bit with the test and noticed that "tbstyle=line" now outputs "All traceback entries are hidden", and "ValueError: bar" will match in all cases. So I removed/updated the relevant if statements.
I don't think this is a bug or anything, just the test not being kept up to date with feature changes.

@jakkdl jakkdl merged commit fa6a232 into pytest-dev:main Dec 12, 2025
33 checks passed
@jakkdl jakkdl deleted the hide_tb branch December 12, 2025 15:15
@nicoddemus nicoddemus added the backport 9.0.x apply to PRs at any point; backports the changes to the 9.0.x branch label Dec 14, 2025
@patchback
Copy link

patchback bot commented Dec 14, 2025

Backport to 9.0.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/9.0.x/fa6a232d529b33f6388dc788d7613cf206c08130/pr-14025

Backported as #14045

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Dec 14, 2025
* don't crash if exceptiongroup has only hidden tb frames

(cherry picked from commit fa6a232)
nicoddemus pushed a commit that referenced this pull request Dec 14, 2025
)

* don't crash if exceptiongroup has only hidden tb frames

(cherry picked from commit fa6a232)

Co-authored-by: John Litborn <11260241+jakkdl@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 9.0.x apply to PRs at any point; backports the changes to the 9.0.x branch bot:chronographer:provided (automation) changelog entry is part of PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IndexError if all post-cut traceback frames are hidden

3 participants